Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

preserve declarations if @internal is mentioned in unrelated comment #57960

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Zzzen
Copy link
Contributor

@Zzzen Zzzen commented Mar 27, 2024

Fixes #57352

@typescript-bot typescript-bot added the For Backlog Bug PRs that fix a backlog bug label Mar 27, 2024
@typescript-bot
Copy link
Collaborator

Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page.

Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up.

Zzzen and others added 2 commits March 28, 2024 01:00
Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
@sandersn sandersn added this to Not started in PR Backlog May 7, 2024
@sandersn sandersn requested a review from rbuckton May 9, 2024 22:30
@sandersn sandersn moved this from Not started to Waiting on reviewers in PR Backlog May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Backlog Bug PRs that fix a backlog bug
Projects
PR Backlog
  
Waiting on reviewers
Development

Successfully merging this pull request may close these issues.

Class member incorrectly stripped from .d.ts output if @internal is mentioned in unrelated comment
3 participants